Daniel Mustieles [Tue, 7 Jan 2014 12:29:40 +0000 (13:29 +0100)]
Updated Spanish translation
Daniel Mustieles [Tue, 7 Jan 2014 12:29:30 +0000 (13:29 +0100)]
Updated Spanish translation
Matthias Clasen [Tue, 7 Jan 2014 03:51:11 +0000 (22:51 -0500)]
GtkHeaderBar: Fix positioning of pack-end widgets
This makes the positioning of pack-start and pack-end
widgets symmetric.
Matthias Clasen [Tue, 7 Jan 2014 03:47:57 +0000 (22:47 -0500)]
Make testtitlebar more versatile
Test adding multiple custom widgets at each end. This exposes
that we're doing it wrong - the pack-end widgets are ordered
in the wrong way.
Baurzhan Muftakhidinov [Tue, 7 Jan 2014 02:33:22 +0000 (08:33 +0600)]
Update Kazakh translation
Matthias Clasen [Tue, 7 Jan 2014 01:35:01 +0000 (20:35 -0500)]
Avoid compiler warnings
Federico Mena Quintero [Mon, 6 Jan 2014 23:15:56 +0000 (17:15 -0600)]
filechooser: Document the settings XML
Matthias Clasen [Mon, 6 Jan 2014 22:05:53 +0000 (17:05 -0500)]
Be more careful when parsing gtk-decoration-layout
When the string is empty, breaking it at : will not give
us two tokens, and we were parsing garbage, leading to crash.
https://bugzilla.gnome.org/show_bug.cgi?id=721635
Ryan Lortie [Fri, 13 Dec 2013 04:31:52 +0000 (23:31 -0500)]
quartz: implement gdk_window_set_shadow_width()
Use the information to allow dragging windows all the way to the top of
the screen (ie: allow the top shadow to go under the menubar).
https://bugzilla.gnome.org/show_bug.cgi?id=720374
Ryan Lortie [Fri, 13 Dec 2013 01:18:01 +0000 (20:18 -0500)]
quartz: fix manual window move
We need to have gdk skip standard processing of events when we are in
manual move in addition to manual resize.
https://bugzilla.gnome.org/show_bug.cgi?id=720357
William Jon McCann [Mon, 6 Jan 2014 18:46:12 +0000 (13:46 -0500)]
tests: add menu alignments to the menubutton test
https://bugzilla.gnome.org/show_bug.cgi?id=720939
William Jon McCann [Mon, 6 Jan 2014 18:11:22 +0000 (13:11 -0500)]
tests: add some space to menubutton test
https://bugzilla.gnome.org/show_bug.cgi?id=720939
William Jon McCann [Sun, 22 Dec 2013 17:43:35 +0000 (12:43 -0500)]
Fix positioning of up menu button popup
Use the menu allocation instead of the request size.
https://bugzilla.gnome.org/show_bug.cgi?id=720939
Giovanni Campagna [Mon, 6 Jan 2014 13:05:59 +0000 (14:05 +0100)]
bloatpad: fix the action associated with the notification
Notifications can only be associated with application actions,
but clear is a window action. Introduce a "clear-all" action
that forwards to clear on all windows.
https://bugzilla.gnome.org/show_bug.cgi?id=721633
Matthias Clasen [Mon, 6 Jan 2014 15:30:15 +0000 (10:30 -0500)]
Add an example for split headers
This shows how to deal with variability in window decorations
with split header bars.
Matthias Clasen [Mon, 6 Jan 2014 15:31:34 +0000 (10:31 -0500)]
GtkHeaderBar: Refresh window buttons when settings change
We already do this when the shell-shows-menu settings changes,
but we forgot to do the same for the decoration layout.
Victor Ibragimov [Mon, 6 Jan 2014 10:18:49 +0000 (15:18 +0500)]
Tajik translation updated
Milo Casagrande [Mon, 6 Jan 2014 08:26:21 +0000 (09:26 +0100)]
[l10n] Updated Italian translation.
Matthias Clasen [Mon, 6 Jan 2014 03:42:09 +0000 (22:42 -0500)]
GtkMenuItem: fix initial accessible selection state
When the accessible is created for an already-selected
menuitem, we were not properly setting the state of the
the accessible. Fix that.
Rico Tzschichholz [Sun, 5 Jan 2014 20:46:40 +0000 (21:46 +0100)]
x11: Include missing public headers to GdkX11 gir build
Simon Feltman [Fri, 16 Aug 2013 10:59:30 +0000 (03:59 -0700)]
Add introspection friendly version of gtk_tree_path_new_from_indices
Add gtk_tree_path_new_from_indicesv which takes an array of
integers with a length. Use "Rename to" annotation to rename the
method as gtk_tree_path_new_from_indices. This is needed because
the original method takes variadic arguments which is not supported
by introspection.
https://bugzilla.gnome.org/show_bug.cgi?id=706119
Rafael Ferreira [Sun, 5 Jan 2014 17:01:54 +0000 (15:01 -0200)]
Updated Brazilian Portuguese translation for UI
Rafael Ferreira [Sun, 5 Jan 2014 17:01:47 +0000 (15:01 -0200)]
Updated Brazilian Portuguese translation for Properties
Matthias Clasen [Sun, 5 Jan 2014 16:56:31 +0000 (11:56 -0500)]
a11y: Fix keybindings in menus
GtkMenuItemAccessible was assuming that an accel label is
always the immediate child of a menu item. It also did not
deal with manually set accels. Fix both of these.
https://bugzilla.gnome.org/show_bug.cgi?id=721086
Matthias Clasen [Sun, 5 Jan 2014 16:12:09 +0000 (11:12 -0500)]
GtkAccelLabel: Add a setter for manual accels
This is needed to keep the a11y keybinding code working.
Matthias Clasen [Sun, 5 Jan 2014 15:54:43 +0000 (10:54 -0500)]
Emit a11y state change notification for color swatches
Improve the color swatch accessible to set a proper role
depending on whether the swatch is selectable or not, and
set the checked atk state when appropriate.
https://bugzilla.gnome.org/show_bug.cgi?id=721048
Matthias Clasen [Sun, 5 Jan 2014 14:19:58 +0000 (09:19 -0500)]
Color chooser: More accessible names
Set the accessible name of the 'custom color' button too.
Matthias Clasen [Sun, 5 Jan 2014 14:11:54 +0000 (09:11 -0500)]
Update a11y tests to match the last change
Color swatches have names now, not descriptions.
Matthias Clasen [Sun, 5 Jan 2014 04:36:12 +0000 (23:36 -0500)]
Nicer nicks for GtkMenuButton properties
Requested in
https://bugzilla.gnome.org/show_bug.cgi?id=720945
Matthias Clasen [Sun, 5 Jan 2014 04:30:28 +0000 (23:30 -0500)]
GtkColorChooserWidget: Set accessible names
As pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721053
we should set the accessible name on the color swatch widgets,
not the descriptions.
Matthias Clasen [Sun, 5 Jan 2014 04:15:40 +0000 (23:15 -0500)]
GtkModelMenuItem: set initial accessible role
The initial state of GtkModelMenuItem is that of a normal menu
item, but the accessible we are using, GtkCheckMenuItemAccessible,
starts out with a role of 'check menu item'. Fix that up by
explicitly setting the initial accessible role.
Matthias Clasen [Sun, 5 Jan 2014 02:57:26 +0000 (21:57 -0500)]
Drop custom colors from a11y output
These were inadvertedly included in the previous commit, but
no custom colors are present in the gnome-continuous test sessions.
Matthias Clasen [Sun, 5 Jan 2014 02:48:19 +0000 (21:48 -0500)]
GtkCheckMenuItemAccessible: Make work with model menu items
GtkModelMenuItem does not emit the ::toggled signal when a radio
item is activated, so listen for property notification for that
property. We still keep the ::toggled signal handler, in order
to not break other uses of check and radio menu items.
https://bugzilla.gnome.org/show_bug.cgi?id=720983
Matthias Clasen [Sun, 5 Jan 2014 02:50:29 +0000 (21:50 -0500)]
GtkModelMenuItem: Add some property notification
GtkModelMenuItem emits no property notification, since none of its
properties are readable. But the toggled property is just a proxy
for GtkCheckMenuItem::active, so we should ensure that property
notification is emitted for the ::active property.
Tristan Van Berkom [Sat, 4 Jan 2014 02:55:41 +0000 (11:55 +0900)]
bgo #721407 - Fix scrolled window to work with template subclasses.
Ensure the hscrollbar & vscrollbar at gtk_scrolled_window_add() time,
this allows one to subclass GtkScrolledWindow with templates and add
children, as this will happen at instance initialization time before
the construct adjustment properties take effect.
Matthias Clasen [Sat, 4 Jan 2014 23:51:23 +0000 (18:51 -0500)]
GtkColorSwatch: Set an accessible role
And update the colorchooser a11y test to a) use GtkColorChooser
instead of the deprecated GtkColorSelection and b) match this
change. Pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721045
Colin Walters [Sat, 4 Jan 2014 22:52:22 +0000 (17:52 -0500)]
testbbox: Remove unused variable
This was the only compiler warning in Continuous for gtk3.
Matthias Clasen [Sat, 4 Jan 2014 22:44:14 +0000 (17:44 -0500)]
GtkInfoBar: Set a more accurate accessible role
And update the a11y tests to match. Pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721042
Matthias Clasen [Sat, 4 Jan 2014 22:36:19 +0000 (17:36 -0500)]
GtkDrawingArea: Set an accessible role
And update the a11y tests to match. Pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721040
Matthias Clasen [Sat, 4 Jan 2014 22:36:10 +0000 (17:36 -0500)]
Trivial formatting fix
Marek Černocký [Sat, 4 Jan 2014 21:57:38 +0000 (22:57 +0100)]
Updated Czech translation
Matthias Clasen [Sat, 4 Jan 2014 20:15:00 +0000 (15:15 -0500)]
Deal with broken _NET_WORKAREA more gracefully
fvwm seems to have problems keeping _NET_WORKAREA in sync with
the number of desktops. Instead of reading garbage, silently use
the full screen as workarea for desktops that are not covered
by the _NET_WORKAREA property.
https://bugzilla.gnome.org/show_bug.cgi?id=698248
Matthias Clasen [Sat, 4 Jan 2014 20:13:51 +0000 (15:13 -0500)]
x11: Avoid a possible memory leak in an error case
If _NET_WORKAREA is set to nonsensical values, we should still
free the memory before returning.
Matthias Clasen [Sat, 4 Jan 2014 19:11:55 +0000 (14:11 -0500)]
Trivial typo fix
Clayton Walker [Wed, 1 Jan 2014 23:38:37 +0000 (00:38 +0100)]
gtk: remove unused GTK_BUTTONBOX_DEFAULT_STYLE from gtkenums.h tests: remove GTK_BUTTONBOX_DEFAULT_STYLE from testbbox.c and fix crashing
https://bugzilla.gnome.org/show_bug.cgi?id=721317
Matthias Clasen [Sat, 4 Jan 2014 18:01:49 +0000 (13:01 -0500)]
GtkNotebook: Fix a problem with tab states
When replacing the tab label, we were not adding the
active-page style class. Fix that.
Matthias Clasen [Sat, 4 Jan 2014 17:40:34 +0000 (12:40 -0500)]
Raleigh: Improve notebook tab theming
The theme was not kept up-to-date when notebook tabs were
recently refactored. Fix it to show some indication of the
active tab.
Matthias Clasen [Sat, 4 Jan 2014 17:12:14 +0000 (12:12 -0500)]
Raleigh: Improve scale theming
Add some visual indication for fine adjustment mode in scales
and scrollbars.
Matthias Clasen [Sat, 4 Jan 2014 16:52:02 +0000 (11:52 -0500)]
Raleigh: Improve titlebar appearance
The default theme wasn't kept up-to-date as the header bar
and titlebar theming was tweaked. Update it to look non-broken,
at least.
Matthias Clasen [Sat, 4 Jan 2014 16:08:29 +0000 (11:08 -0500)]
GtkAboutDialog: be consistent about link activation
As pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721394,
links in the credits part were not going through the
::link-activated signal. Fix that.
Matthias Clasen [Sat, 4 Jan 2014 15:57:46 +0000 (10:57 -0500)]
Trivial grammar fix
Matthias Clasen [Sat, 4 Jan 2014 15:47:22 +0000 (10:47 -0500)]
GtkLabelAccessible: reorganize the code
Separate the GtkLabelAccessible, GtkLabelAccessibleLink and
GtkLabelAccessibleLinkImpl implementations more clearly.
Matthias Clasen [Sat, 4 Jan 2014 14:43:23 +0000 (09:43 -0500)]
GtkLabelAccessible: update state of links
This commit makes the LinkImpl children of label accessibles
inherit all their state, except for focused and visited, which
are handled directly.
https://bugzilla.gnome.org/show_bug.cgi?id=721406
Matthias Clasen [Sat, 4 Jan 2014 03:02:16 +0000 (22:02 -0500)]
a11y tests: Add a test for multi-link labels
Matthias Clasen [Sat, 4 Jan 2014 03:59:37 +0000 (22:59 -0500)]
GtkLabelAccessible: Implement AtkHypertext
This commit makes the label accessible implement AtkHypertext,
which returns a AtkHyperlink object for each link in the text.
At the same time, add AtkHyperlinkImpl objects as children
to the label accessible.
Also some private API to indicate that links have changed, and
call that from GtkLabel when needed.
Adjust expected output of the affected a11y tests.
https://bugzilla.gnome.org/show_bug.cgi?id=721410
https://bugzilla.gnome.org/show_bug.cgi?id=721421
Matthias Clasen [Sat, 4 Jan 2014 02:00:44 +0000 (21:00 -0500)]
GtkLabel: Add private api for links
This exposes enough information to implement the AtkHyperText
interface.
More private link api
Matthias Clasen [Sat, 4 Jan 2014 01:44:21 +0000 (20:44 -0500)]
Add a gtklabelprivate.h header
For now, move the few private functions from gtklabel.h here.
Matthias Clasen [Sat, 4 Jan 2014 01:34:22 +0000 (20:34 -0500)]
a11y tests: Prepare to dump AtkHypertext
This will be used for links in labels soon.
As a side-effect, the output for AtkHyperlinkImpl changes
slightly. Update expected output accordingly.
Matthias Clasen [Sat, 4 Jan 2014 02:58:31 +0000 (21:58 -0500)]
GtkLinkButtonAccessible: fix broken AtkHyperlink implementation
The get_end_index implementation was casting the accessible
to AtkText even though GtkLinkButtonAccessible does not
implement this interface. This did not show up in the a11y
tests because the they were not dumping the affected AtkHyperlinkImpl
properties. Oops.
Matthias Clasen [Fri, 3 Jan 2014 21:19:54 +0000 (16:19 -0500)]
Deprecated the decoration-button-layout style property
This is now done via a setting + regular property, for
better control, and the style property is ignored.
Tong Hui [Sat, 4 Jan 2014 09:12:24 +0000 (17:12 +0800)]
Update Chinese simplified translation
Ryan Lortie [Fri, 27 Dec 2013 03:45:03 +0000 (22:45 -0500)]
GtkMenuTracker: tweak separator logic
Ignacio Casal Quinteiro reported a problem whereby an empty section at
the start of a menu has a separator placed after it. This was caused by
the implementation of the logic that separators should be inserted at
the top of all non-empty sections that are not the first section. This
logic is obviously incorrect in the case that the first section is empty
(in which case we would not expect to see a separator at the top of the
second section).
Change the logic so that we only insert separators when we see a
non-zero number of actual items in the menu before us.
https://bugzilla.gnome.org/show_bug.cgi?id=721119
Aurimas Černius [Fri, 3 Jan 2014 21:28:45 +0000 (23:28 +0200)]
Updated Lithuanian translation
Rafael Ferreira [Fri, 3 Jan 2014 07:46:30 +0000 (05:46 -0200)]
Updated Brazilian Portuguese translation for UI
Rafael Ferreira [Fri, 3 Jan 2014 07:43:56 +0000 (05:43 -0200)]
Updated Brazilian Portuguese translation for Properties
Ryan Lortie [Thu, 2 Jan 2014 17:00:10 +0000 (12:00 -0500)]
configure: use CFLAGS, etc. _FOR_BUILD as well
When not doing cross-builds, use the values of CFLAGS, CPPFLAGS and
LDFLAGS as the default value for CFLAGS_FOR_BUILD, CPPFLAGS_FOR_BUILD
and LDFLAGS_FOR_BUILD, respectively.
This avoids having to manually specify these variables in order to get
extract-strings to build properly.
This should really be handled by ax_prog_cc_for_build.m4. That has been
reported upstream. This is a workaround for now.
https://bugzilla.gnome.org/show_bug.cgi?id=721346
Colin Walters [Thu, 2 Jan 2014 23:40:17 +0000 (18:40 -0500)]
window: Document GtkWindow::set-focus
Not sure why this was never documented.
Giovanni Campagna [Wed, 1 Jan 2014 15:53:31 +0000 (16:53 +0100)]
GtkApplicationX11: read the startup ID from the platform data
If the platform data passed with actions and activations includes
a startup notify ID, we should read it and pass it down to GDK.
This ensures that the right startup notify is completed after the
signal emission, and that the user time of the GdkDisplay is properly
updated (which in turn makes sure the windows are not subjected
to focus-stealing-prevention)
https://bugzilla.gnome.org/show_bug.cgi?id=721304
Tong Hui [Wed, 1 Jan 2014 11:01:29 +0000 (19:01 +0800)]
Update Chinese simplified translation for gtk-properties
Wylmer Wang [Wed, 1 Jan 2014 10:32:10 +0000 (18:32 +0800)]
Revert "Update Chinese simplified translation"
This reverts commit
410d09ee4ee4ab0ad4440d07f09d532a52e6acc6.
Undo mistaken push of Chinese simplified translation
Tong Hui [Wed, 1 Jan 2014 09:53:58 +0000 (17:53 +0800)]
Update Chinese simplified translation
Christian Kirbach [Tue, 31 Dec 2013 17:08:26 +0000 (18:08 +0100)]
Updated German translation
Daniel Mustieles [Mon, 30 Dec 2013 10:58:05 +0000 (11:58 +0100)]
Updated Spanish translation
Daniel Mustieles [Mon, 30 Dec 2013 10:57:52 +0000 (11:57 +0100)]
Updated Spanish translation
Chun-wei Fan [Mon, 30 Dec 2013 05:17:43 +0000 (13:17 +0800)]
Rework the Visual Studio 2010 Projects a bit
Add a utility project to get config.h and gdkconfig.h from their *.h.win32
(or win32_broadway, if applicable) counterparts, using custom build rules,
so that these "generated" files can also be removed on clean and
"regenerated" upon update. This also enables the removal of configs in
certain projects that isn't really needed as a result.
Also update and merge the projects and property sheets to include a single
property sheet that it needs, which will then in turn include the other
property sheets that is needed, so that things are cleaner.
Chun-wei Fan [Mon, 30 Dec 2013 04:07:34 +0000 (12:07 +0800)]
Rework the Visual Studio 2008 Projects a bit
Add a utility project to get config.h and gdkconfig.h from their *.h.win32
(or win32_broadway, if applicable) counterparts, using custom build rules,
so that these "generated" files can also be removed on clean and
"regenerated" upon update. This also enables the removal of configs in
certain projects that isn't really needed as a result.
Also update and merge the projects and property sheets to include a single
property sheet that it needs, which will then in turn include the other
property sheets that is needed, so that things are cleaner.
Updates to the Visual Studio 2010 projects will follow later.
Fran Diéguez [Sun, 29 Dec 2013 23:11:25 +0000 (00:11 +0100)]
Updated Galician translations
Fran Diéguez [Sun, 29 Dec 2013 22:59:30 +0000 (23:59 +0100)]
Updated Galician translations
Benjamin Otte [Sat, 28 Dec 2013 14:41:13 +0000 (15:41 +0100)]
treeview: Track area in cell that editable occupies
With multiple renderers per cell, editables shouldn't occupy the whole
cell's area.
https://bugzilla.gnome.org/show_bug.cgi?id=710315
Dimitris Spingos [Thu, 26 Dec 2013 16:40:01 +0000 (18:40 +0200)]
Updated Greek translation po-properties
Chun-wei Fan [Thu, 26 Dec 2013 08:23:14 +0000 (16:23 +0800)]
Update the gtk3-demo Visual C++ Projects
Add the sources of the headerbar and event_axes demos to the list of files
to build.
Chun-wei Fan [Thu, 26 Dec 2013 08:15:10 +0000 (16:15 +0800)]
gtk/gtkapplication.c: Don't Include unistd.h Unconditionally
This header, which is not universally available, is accidently made to be
included unconditionally during the refactoring of gtkapplication.c,
so restore the #ifdef check.
Milo Casagrande [Tue, 24 Dec 2013 08:03:52 +0000 (09:03 +0100)]
[l10n] Updated Italian translation.
Piotr Drąg [Sun, 22 Dec 2013 19:30:49 +0000 (20:30 +0100)]
Updated POTFILES.skip
Matthias Clasen [Sun, 22 Dec 2013 02:58:24 +0000 (21:58 -0500)]
Redo header bar decorations once more
Applications need a way to fix or adapt the decoration layout,
for situations like split header bars. Setting the layout from
the theme with a style property did not offer a good way to do
this, and the ::show-close-button property does not provide
fine-grained control.
To improve the situation, move the layout string to a property of
GtkHeaderBar which is backed by a setting. This allows platforms to
set a default button layout independent of the theme, while applications
can override the default.
The style GtkWindow style property is now deprecated and ignored.
Matthias Clasen [Fri, 20 Dec 2013 20:15:31 +0000 (15:15 -0500)]
Fix a few warnings
Matthias Clasen [Fri, 20 Dec 2013 18:24:12 +0000 (13:24 -0500)]
Allow close buttons on dialogs
It was a long-standing problem for some users of GtkDialog that
there is no guaranted way to close it. So, lets show a close
button.
Matthias Clasen [Fri, 20 Dec 2013 15:58:32 +0000 (10:58 -0500)]
Improve menubutton a11y
The button now claims its menu as a child for a11y purposes,
which makes it possible for ATs to see it when the navigate
the tree top-down.
Update the a11y test to match.
Matthias Clasen [Fri, 20 Dec 2013 14:06:33 +0000 (09:06 -0500)]
Add a menubutton to the a11y tests
Matthias Clasen [Fri, 20 Dec 2013 14:05:43 +0000 (09:05 -0500)]
Set an accessible name on menu buttons
These commonly have just an image in them, so lets at least
call them 'Menu'.
Dimitris Spingos [Fri, 20 Dec 2013 13:27:53 +0000 (15:27 +0200)]
Updated Greek translation
Stas Solovey [Thu, 19 Dec 2013 19:10:26 +0000 (23:10 +0400)]
Updated Russian translation
Matthias Clasen [Thu, 19 Dec 2013 16:22:02 +0000 (11:22 -0500)]
GtkPlacesSidebar: Don't show desktop if its home
We go to extra length to set the desktop_uri to NULL when
desktop == home, but then we were adding the (non-functional)
place item anyway. Don't do that.
Matthias Clasen [Thu, 19 Dec 2013 13:16:30 +0000 (08:16 -0500)]
GtkPlacesSidebar: avoid a crash
Be more careful when comparing uris during DND - they may
be NULL.
https://bugzilla.gnome.org/show_bug.cgi?id=720264
Jonh Wendell [Mon, 16 Dec 2013 19:31:50 +0000 (17:31 -0200)]
cell-renderer: Check for NULL return in vfunc start_editing()
This virtual function may return NULL, so, let's check if that's
the case and exit the function apropriately.
https://bugzilla.gnome.org/show_bug.cgi?id=720554
Marek Kasik [Mon, 16 Dec 2013 15:44:58 +0000 (16:44 +0100)]
printing: Don't leak http connection to CUPS
Close http connection to CUPS after execution
of request which created it.
https://bugzilla.gnome.org/show_bug.cgi?id=720535
Fran Diéguez [Thu, 19 Dec 2013 00:41:14 +0000 (01:41 +0100)]
Updated Galician translations
Marek Kasik [Wed, 18 Dec 2013 11:23:28 +0000 (12:23 +0100)]
gdkwindow: Don't add the same window to "update_windows" twice
This prevents passing of such window to another GMainLoop.
https://bugzilla.gnome.org/show_bug.cgi?id=711552
Matthias Clasen [Wed, 18 Dec 2013 04:26:16 +0000 (23:26 -0500)]
GtkColorSelection: Don't leak a context menu
This was pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=720401
Matthias Clasen [Wed, 18 Dec 2013 04:25:21 +0000 (23:25 -0500)]
Fix a compiler warning